Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

graphPeaks (recreate PR after merging) #23

Closed
wants to merge 25 commits into from

Conversation

tnaake
Copy link
Collaborator

@tnaake tnaake commented Oct 14, 2019

As written here, here is the new PR against the devel branch for the graphPeaks function.

@sgibb sgibb self-requested a review October 14, 2019 11:50
@sgibb sgibb self-assigned this Oct 14, 2019
@sgibb sgibb added the enhancement New feature or request label Oct 14, 2019
@jorainer
Copy link
Member

I really like the functionality that you add here, only, I think we have to fix this PR. I don't know what would be best, closing this PR, try to sync your branch to the master (no longer the devel branch) and create it again? @sgibb any suggestions?

@sgibb sgibb mentioned this pull request Oct 25, 2019
@sgibb
Copy link
Member

sgibb commented Oct 25, 2019

Closing this in favour of #25. @tnaake thanks for your contribution please find an explanation in my PR.

@sgibb sgibb closed this Oct 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants